The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
release-back	Ready	Preview, Comment	Mar 21, 2026 10:37am
release-test	Ready	Preview, Comment	Mar 21, 2026 10:37am

코드 리뷰 — Phase 1 잔여물 정리 + Phase 2 entities 레이어 구축

Claude Code (Pro) 로컬 리뷰 | 2026-03-21

✅ 잘 된 점

re-export 하위 호환 전략
src/api/index.ts → @/shared/api re-export 패턴으로 17개 도메인 파일을 한 줄도 건드리지 않고 axiosInstance를 FSD 안으로 이전했다. 대규모 리팩토링에서 점진적 이전의 교과서적 예시.

레이어 위반 선제 해소
Filters (hooks → entities/search), IListParams (hooks → entities/community) 두 타입을 발견하고 즉시 이전 + re-export 처리. entities가 hooks를 import하는 구조였다면 Phase 3에서 순환 참조 가능성이 있었다.

불필요 import 정리
bookmark/api.ts에서 사용하지 않던 ITripList, daysAgo, dayjs, authStore 4개를 제거. 원본 코드의 기술 부채를 이전 과정에서 함께 해소.

⚠️ 개선 필요 사항

[중요] retryCount가 모듈 전역 변수 — 요청 간 상태 공유

// src/shared/api/axiosInstance.ts:17
let retryCount = 0; // ← 모듈 레벨 전역 변수

현재 retryCount가 모듈 레벨에 선언되어 있어 모든 API 요청이 카운터를 공유한다. 예를 들어 요청 A가 401을 3번 받고, 이후 요청 B가 401을 받으면 이미 카운터가 3인 상태로 시작한다. 리셋 로직도 없다.

원본 코드부터 있던 버그이므로 Phase 2 범위는 아니지만, Phase 3 auth feature 작업 시 수정 필요.

// 권장 방향 (Phase 3에서):
// originalRequest에 _retryCount를 붙여 요청별로 추적
async (error) => {
  const originalRequest = error.config;
  originalRequest._retryCount = (originalRequest._retryCount ?? 0) + 1;
  if (originalRequest._retryCount > MAX_RETRY_COUNT) { ... }
}

[중요] getHomeUserProfile — 잘못된 파라미터 사용

// src/entities/trip/api.ts:58
export const getHomeUserProfile = async (accessToken: string) => {
  const response = await axiosInstance.get(`/api/profile/me?userNumber=${accessToken}`);
  //                                                        ^^^^^^^^^^^^^^^^^^^^^^^^
  //                                          accessToken을 userNumber로 쿼리스트링에 전달 (버그)
};

accessToken을 userNumber 쿼리 파라미터로 전달하는 것은 명백한 버그. 원본 코드(api/home.ts)부터 있던 문제이며, 함수명도 getHomeUserProfile이어서 myPage/getMyPage와 역할이 겹친다. Phase 3에서 auth feature 작업 시 제거 or 수정 검토 필요.

[경미] entities/user/api.ts — 에러 처리 불일치

같은 파일 내에서 에러 처리 방식이 혼재한다.

// getUserTravelLog: RequestError throw (일관성 있음)
} catch (err: any) {
  throw new RequestError(err);
}

// getUser: console.error만 하고 undefined 반환 (불일관)
} catch (error: any) {
  console.error(error);
  // throw 없음 → 호출부에서 null/undefined 처리 필요
}

// kakaoLogin, googleLogin, naverLogin: console.error만 (불일관)

원본 코드의 패턴을 유지한 것이지만 Phase 3 auth feature 작업 시 RequestError throw로 통일 권장.

[경미] tripDetail/api.ts — entities 간 직접 import

// src/entities/tripDetail/api.ts:4
import { UpdateTripReqData } from '@/entities/trip';

FSD 공식 가이드에서는 같은 레이어(entities) 내 교차 import를 금지한다. UpdateTripReqData는 tripDetail이 아닌 trip entity에 속해 있어 현재 이 구조가 불가피하지만, Phase 3에서 두 가지 해결책을 고려:

1. UpdateTripReqData를 shared/types/로 올려 공유
2. tripDetail의 update 기능을 trip entity로 통합

[경미] shared/ui/index.ts — export * 이름 충돌 잠재 위험

export * from './button';
export * from './input';
// ...

현재는 문제없지만, 향후 카테고리 간 동일한 이름이 생기면 export * 충돌로 빌드 에러가 발생한다. 규모가 커지면 named re-export 방식도 고려.

📊 수치 요약

🎯 Phase 3 전 처리 권장 항목